-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve user feedback and speed-up interlis import qgep #141
Conversation
Should improve (I have added some logger info that it is still processing) #119 - not yet best user experience, but already better |
It looks like the triggers are re-enable before the data is imported - the wizard is still open - see logging on screen shot. @domi4484 Is this a similar error as you have found in TEKSI wastewater? https://github.com/teksi/wastewater/pull/211/files How could the TEKSI improvment be backported to qgep? |
@sjib Yes it seems similar. Basically you need to re-enable the triggers only after the dialog has been acceptedd/rejected. By re-thinking on this we could have an issue (in TEKSI as well) if someone kills QGIS before the trigger are re-enabled they will remain disabled forever. |
This speeds up the import greatly!
With that the triggers are enabled after the import - this speeds up the import greatly (e.g. Testdata K from @urskaufmann in 3 minutes instead of 30 minutes or more). This needs these datamodel changes 1.6.2 to work properly! |
|
Take adaptions to other models, not just qgepdss
@ponceta Ready for review and then merging - needs these QGEP/datamodel#235 to work properly! |
Datamodel - see QGEP/datamodel@1c90791